Skip to content

Conversation

@SaraDPH
Copy link
Contributor

@SaraDPH SaraDPH commented Jul 25, 2025

Proposed changes

Unreleased product version (optional)

Related issues (optional)

Always the spaces!!!! Why can't it just ignore it
@pulumi-bot
Copy link
Collaborator

@SaraDPH
Copy link
Contributor Author

SaraDPH commented Jul 25, 2025

Note to self: re-organize the list

@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

@SaraDPH
Copy link
Contributor Author

SaraDPH commented Dec 2, 2025

@claude where is the trailing space? Review!

@claude
Copy link
Contributor

claude bot commented Dec 2, 2025

Claude finished @SaraDPH's task —— View job


Review Complete

  • Read the changed blog post file
  • Check git diff to see exact changes
  • Search for trailing spaces
  • Review content for style guide compliance
  • Provide comprehensive review feedback

Issues Found

1. Trailing Spaces (10 occurrences)

Found trailing whitespace on the following lines:

  • Line 74: End of paragraph in section 2
  • Lines 108-112: All bullet points in section 4 (Enterprises Rebuild Their Cloud Foundations)
  • Lines 118-121: All bullet points in subsection "Pulumi users increasingly rely on"

According to AGENTS.md, you can fix these with:

sed -i '' 's/[[:space:]]*$//' content/blog/future-cloud-infrastructure-10-trends-shaping-2024-and-beyond/index.md

2. Duplicate Line (Lines 89-91)

The text "Modern cloud teams use:" appears twice consecutively in section 3:

Modern cloud teams use:

Modern cloud teams use:

- **Infrastructure as Code** for consistent...

One of these should be removed (content/blog/future-cloud-infrastructure-10-trends-shaping-2024-and-beyond/index.md:91).


Style Guide Compliance

The content otherwise follows the style guide well:

  • ✅ H2 headings use sentence case (except for acronyms like "IaC", "IDPs", "AIOps" which are correct)
  • ✅ Proper capitalization of product names (Pulumi IaC, Pulumi ESC, Pulumi Insights, etc.)
  • ✅ No heading punctuation
  • ✅ Code elements properly wrapped in backticks
  • ✅ Links are descriptive
  • ✅ Paragraphs are well-formatted

Content Quality

The annual update significantly improves the blog post:

  • Strong focus on 2026 trends with updated data and quotes
  • Excellent use of current statistics and industry reports
  • Good integration of AI/ML themes throughout
  • Clear connections to Pulumi products and capabilities
  • Well-structured progression from hyperscaler investments to enterprise adoption patterns

Summary

Required fixes:

  1. Remove 10 trailing spaces
  2. Remove duplicate "Modern cloud teams use:" line

Recommendation: Run the sed command above to fix trailing spaces, then manually remove the duplicate line at line 91.


@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

…4-and-beyond/index.md


updating links and Portal to Platform
@pulumi-bot
Copy link
Collaborator

Update workshop
Add K8s operate v2.3.0
@pulumi-bot
Copy link
Collaborator

Adding a link to the new landing page
Adding the 10 trends list
@SaraDPH
Copy link
Contributor Author

SaraDPH commented Dec 3, 2025

ok @claude do a last check

@claude
Copy link
Contributor

claude bot commented Dec 3, 2025

Claude finished @SaraDPH's task —— View job


Final PR Check ✅

  • Read the current state of the blog post
  • Check git diff to see all changes
  • Verify trailing spaces are fixed
  • Verify duplicate line is fixed
  • Review style guide compliance
  • Check for any remaining issues
  • Provide final review feedback

Review Summary

Great work! All previously identified issues have been fixed. The PR is ready to merge.

✅ Issues Fixed

  1. Trailing spaces: All 10 trailing whitespace issues have been removed
  2. Duplicate line: The duplicate "Modern cloud teams use:" line has been removed
  3. Style guide compliance: All content follows the style guide properly

✅ Content Quality

The annual update is comprehensive and well-executed:

  • Updated year references: Successfully updated from 2025 to 2026 throughout
  • Current data and statistics: Includes recent 2025 data, quotes, and industry reports
  • Strong AI/ML focus: AI infrastructure, AI agents, and AI-native cloud architectures are well integrated
  • Updated product references: Properly references Pulumi Neo (replacing Copilot references)
  • Table of contents: Added clear navigation with anchor links
  • New meta image: Added "2026-cloud-trends -ai-idp-security.png" (note: filename has a space after "trends")

✅ Style Guide Compliance

  • Proper heading case (H1 title case, H2+ sentence case)
  • Correct capitalization of product names
  • No heading punctuation
  • Code elements properly wrapped
  • Links are descriptive
  • American English spelling
  • File ends with newline

Minor Note (Non-blocking)

The meta_image filename "2026-cloud-trends -ai-idp-security.png" has a space after "trends". While this works fine (the file exists), it's generally better practice to use hyphens instead of spaces in filenames. This is cosmetic only and doesn't need to block the merge.

Changed Files Review

  1. index.md (main blog post): ✅ All good
  2. 2026-cloud-trends -ai-idp-security.png (new image): ✅ File exists and is referenced correctly
  3. beyond-yaml-kubernetes-2026-automation-era/index.md: ✅ Minor updates, no issues found

Recommendation: This PR is ready to merge! All critical issues have been resolved.

Updating copy based on Scott's feedback.
@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

Space it! no one likes blocks of text
Why can't you ignore a silly 1 space, sheesh.
@pulumi-bot
Copy link
Collaborator

@SaraDPH SaraDPH merged commit fc35bdd into master Dec 3, 2025
8 checks passed
@SaraDPH SaraDPH deleted the 10-trends-annual-update branch December 3, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants